Skip to content

Conversation

matthewdale
Copy link
Collaborator

GODRIVER-3538

Summary

Remove the two remaining Atlas Serverless tests from the internal/cmd/testatlas package.

Background & Motivation

@matthewdale matthewdale requested a review from a team as a code owner October 3, 2025 15:35
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes two Atlas Serverless test variants from the test atlas package as part of GODRIVER-3538. The change eliminates serverless-specific test cases while maintaining the existing Atlas test coverage for other variants.

  • Removes "Atlas with serverless" test case
  • Removes "Atlas with srv file on serverless" test case

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@matthewdale matthewdale added review-priority-normal Medium Priority PR for Review: within 1 business day ignore-for-release labels Oct 3, 2025
Copy link
Contributor

🧪 Performance Results

Commit SHA: b11a6b8

The following benchmark tests for version 68dfed31bf90a50007dedd66 had statistically significant changes (i.e., |z-score| > 1.96):

Benchmark Measurement % Change Patch Value Stable Region H-Score Z-Score
BenchmarkSingleFindOneByID total_time_seconds -9.3632 1.0413 Avg: 1.1488
Med: 1.1561
Stdev: 0.0482
0.7524 -2.2318

For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch.

Copy link
Contributor

API Change Report

No changes found!

@matthewdale matthewdale merged commit 3887759 into mongodb:master Oct 3, 2025
34 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore-for-release review-priority-normal Medium Priority PR for Review: within 1 business day

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants